New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RO359B] platform added #4302
[RO359B] platform added #4302
Conversation
targets/targets.json
Outdated
"inherits": ["MCU_K24F1M"], | ||
"detect_code": ["1022"], | ||
"release_versions": ["2", "5"], | ||
"extra_labels_add": ["RO359B"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not needed, and discouraged, to add yourself to extra_labels.
targets/targets.json
Outdated
"MCU_K24F1M": { | ||
"core": "Cortex-M4F", | ||
"supported_toolchains": ["ARM", "GCC_ARM", "IAR"], | ||
"extra_labels": ["Freescale", "MCUXpresso_MCUS", "KSDK2_MCUS", "MCU_K24F", "MCU_K24F1M", "KPSDK_MCUS", "KPSDK_CODE", "FLASH_CMSIS_ALGO"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment on this line: you don't need to add yourself to extra_labels, and you should not.
@theotherjimmy fixed pointed things on your review, and ran tests again. |
tools changes and targets.json look fine. |
/morph export-build |
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 32 All exports and builds passed! |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Thanks! @toyowata @0xc0170 @theotherjimmy |
Description
Adding a new platform RO359B, Wi-Fi module that uses K24F. Re-opened, older one is #4281
Now directory stracture are following, and using inherits on target.json
Status
READY
Migrations
NO
Steps to test or reproduce
To test this platform with ci-test-shield, pelase use following patch. PWM test need some wiring and this patch.
https://gist.github.com/ytsuboi/9cc9ae551e4775994a89eb88700105c9
Test results. ( I didn't use above test patch for this test. )
RO359B_test_ARM_20170511.txt
RO359B_test_GCC_ARM_20170511.txt
RO359B_test_IAR_20170511.txt
CC @toyowata @0xc0170 @theotherjimmy